Skip to content

[18.0][ADD] edi_storage_component_oca#228

Open
ArnauCForgeFlow wants to merge 1 commit intoOCA:18.0from
ForgeFlow:18.0-add-edi_storage_component_oca
Open

[18.0][ADD] edi_storage_component_oca#228
ArnauCForgeFlow wants to merge 1 commit intoOCA:18.0from
ForgeFlow:18.0-add-edi_storage_component_oca

Conversation

@ArnauCForgeFlow
Copy link
Copy Markdown

@ArnauCForgeFlow ArnauCForgeFlow commented Jan 30, 2026

Glue module between edi_storage_oca and edi_component_oca.

It implements the EDI processing listener using the OCA component framework, allowing custom behaviors to be executed when EDI records are processed.

@ArnauCForgeFlow ArnauCForgeFlow force-pushed the 18.0-add-edi_storage_component_oca branch 3 times, most recently from a7f074e to 6295c61 Compare January 30, 2026 16:21
@ArnauCForgeFlow ArnauCForgeFlow force-pushed the 18.0-add-edi_storage_component_oca branch from 6295c61 to 8c2fda2 Compare March 18, 2026 15:02
@ArnauCForgeFlow
Copy link
Copy Markdown
Author

@etobella @simahawk

@ArnauCForgeFlow
Copy link
Copy Markdown
Author

Friendly reminder @etobella @simahawk

Copy link
Copy Markdown
Contributor

@LoisRForgeFlow LoisRForgeFlow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It restores a behavior lost in the migration as mentioned here #226.

LGTM 👍

@ArnauCForgeFlow ArnauCForgeFlow force-pushed the 18.0-add-edi_storage_component_oca branch from 8c2fda2 to 85f4fbe Compare March 26, 2026 10:44
@ArnauCForgeFlow ArnauCForgeFlow force-pushed the 18.0-add-edi_storage_component_oca branch from 85f4fbe to 13bb85c Compare March 26, 2026 10:46
@LoisRForgeFlow
Copy link
Copy Markdown
Contributor

@etobella @simahawk good to merge?

Copy link
Copy Markdown
Contributor

@simahawk simahawk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ArnauCForgeFlow coming from #232 I'm a bit confused: if that work is completed we do not need this. Isn't it?
And w/o this, edi_storage is kind of broken....

@@ -0,0 +1,78 @@
# Copyright 2026 ForgeFlow S.L. (https://www.forgeflow.com)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

original copyright not respected

@@ -0,0 +1 @@
- Arnau Cruz \<[email protected]\>
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

original contributors not respected

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants